-
Notifications
You must be signed in to change notification settings - Fork 0
Add v2.1 BGC data #56
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
To do:
|
forsyth2
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@BunnyVon (and cc'ing @chengzhuzhang as a second reviewer for this PR) I have an initial web page -- preview at https://portal.nersc.gov/cfs/e3sm/forsyth/data_docs_56/html/v2.1/BGC/simulation_data/simulation_table.html:
Remaining questions:
- Do any of the 3 simulations have/need ESGF links?
(see comments for remainder)
run_scripts/v2.1/original/run.CBGCv2.r05.chrysalis.I20TRGSWCNP.sh
Outdated
Show resolved
Hide resolved
| v2.1, WaterCycle, LR, Historical, v2_1.LR.historical_0251, chrysalis, , historical, 4, cmip_only, cels.anl, | ||
| v2.1, WaterCycle, LR, Historical, v2_1.LR.historical_0301, chrysalis, , historical, 5, cmip_only, cels.anl, | ||
| model_version, group, resolution, category, simulation_name, machine, checksum, experiment, ensemble_num, cmip_only, node, | ||
| v2.1, BGC, LR, SPINUP, CBGCv2.r05.chrysalis.I20TRGSWCNP, chrysalis, , , , , , |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- So, we still want this data grouped with v2.1 even though the cases are prefixed with
v2(notv2.1), and this simulation in particular seems to use a completely different naming convention?
All runscripts are placed in chrysalis:/lcrc/globalscratch/ac.sfeng1/archive/runscript |
9a7c913 to
4f488a0
Compare
forsyth2
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@chengzhuzhang -- @BunnyVon would like to have this merged by end of day Friday (3/14). Please review if possible, and if you don't have time, I can go ahead and merge this (after some changes detailed below); we can always update the web pages later.
@BunnyVon added more data to one of the archives, so I will rerun generate_html.bash before merging so the data size column gets updated.
Answers to my remaining questions:
I think this looks great by reviewing the preview page. |
|
Thanks @chengzhuzhang! |
|
Ok, I made my updates and will now merge this. The only remaining issue is that the "NERSC HPSS web interface" link works for Water Cycle, but not for BGC: "NERSC HPSS web interface". I believe the fix for that is to make the simulation directories world-readable. |
|
https://portal.nersc.gov/archive/home/projects/e3sm/www/BGC/E3SMv2_1 works now, looks like the directory permissions change worked. |
|
@chengzhuzhang I just noticed that @BunnyVon's link is https://portal.nersc.gov/cfs/e3sm/forsyth/data_docs_56/html/v2.1/BGC/index.html, which is in my individual directory. Furthermore, the page doesn't seem to exist where it should in the real docs: https://docs.e3sm.org/e3sm_data_docs/_build/html/v2.1/index.html I thought merging the PR here triggered a documentation build, so I'm confused as to why docs.e3sm.org isn't updated. This PR more or less matches #50, which did manage to update docs.e3sm.org... |
|
the gh-page is not built with this change:https://github.com/E3SM-Project/e3sm_data_docs/tree/gh-pages |
|
Yes, it looks like the build did fail: https://github.com/E3SM-Project/e3sm_data_docs/actions/runs/13861015009/job/38789265167#step:1:37: But I'm not seeing a good way to re-run it. Running the workflow on
@tomvothecoder Have you run into this? I'll keep digging into it, but figured I'd ask as well. |
No I have not, but probably because I updated build workflows to Try opening a new PR to update the build work flow to use EDIT: Nevermind I just saw you tried that in #58. |
|
Thanks @tomvothecoder, it looks like #58 got us working -- https://docs.e3sm.org/e3sm_data_docs/_build/html/v2.1/BGC/index.html |



Resolves #55. Add v2.1 BGC data. The link, once merged, will be
https://docs.e3sm.org/e3sm_data_docs/_build/html/v2.1/BGC.HPSS path for data:
/home/projects/e3sm/www/BGC/E3SMv2_1Simulation metadata: https://acme-climate.atlassian.net/wiki/spaces/ED/pages/2766340117/V2+Simulation+Planning